-
Notifications
You must be signed in to change notification settings - Fork 860
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use Invalid payload attributes error code #6582
Use Invalid payload attributes error code #6582
Conversation
Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
|
@jflo I did not yet get a chance to verify this fixes those hive tests |
Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
all the "Invalid Payload" tests are now passing off this branch. Running the full engine-cancun suite locally now to verify no regressions. |
… cancun Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
… into invalid-payload-attributes
… cancun Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
ok ready for review for real this time - only 1 remaining hive test failing (was failing before) - "Invalid Missing Ancestor Syncing ReOrg, StateRoot, EmptyTxs=False, CanonicalReOrg=False, Invalid P9 (Cancun)" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
this test is actually passing when we look online - it just fails locally. |
* invalid payload error code * use non-zero timestamp * use invalidPayloadAttributes error for V2 if non-null beacon root pre cancun Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com> --------- Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
* invalid payload error code * use non-zero timestamp * use invalidPayloadAttributes error for V2 if non-null beacon root pre cancun Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com> --------- Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com> Signed-off-by: amsmota <antonio.mota@citi.com>
* invalid payload error code * use non-zero timestamp * use invalidPayloadAttributes error for V2 if non-null beacon root pre cancun Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com> --------- Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com> Signed-off-by: amsmota <antonio.mota@citi.com>
Return "invalid payload attributes" instead of "invalid params" if the payload attributes are, in fact, invalid
fixes #6554